|
c9bf5465
|
2024-08-12T11:47:16
|
|
Reland "Remove the linkage-specification on `main`"
This is a reland of commit b60dc9d59aad03e641b86431c277c877d4ca9d7a
Original change's description:
> Remove the linkage-specification on `main`
>
> According to the standard, the main function should have no
> linkage-specification [1]. New version of clang will enforce
> this via a warning [2] which causes build failure if building
> with `-Werror` which treats warnings as error.
>
> Remove the superfluous linkage-specification to fix the error.
>
> [1]: https://eel.is/c++draft/basic.start#main-3
> [2]: https://github.com/llvm/llvm-project/pull/101853
>
> Fixed: chromium:358427300
> Change-Id: I541166cbd122c7e296ecdd2b5a9f9a107fe82872
> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5782349
> Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Fixed: chromium:358427300
Change-Id: I380fdf74dc2a056f94af35c5c6c566a88d750c7d
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5785953
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Yuly Novikov <ynovikov@chromium.org>
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
|
|
f0465cac
|
2024-08-12T15:46:14
|
|
Revert "Remove the linkage-specification on `main`"
This reverts commit b60dc9d59aad03e641b86431c277c877d4ca9d7a.
Reason for revert: Fails to roll into Chromium
https://chromium-review.googlesource.com/c/chromium/src/+/5782182
https://cr-buildbucket.appspot.com/build/8739800008401884241
Original change's description:
> Remove the linkage-specification on `main`
>
> According to the standard, the main function should have no
> linkage-specification [1]. New version of clang will enforce
> this via a warning [2] which causes build failure if building
> with `-Werror` which treats warnings as error.
>
> Remove the superfluous linkage-specification to fix the error.
>
> [1]: https://eel.is/c++draft/basic.start#main-3
> [2]: https://github.com/llvm/llvm-project/pull/101853
>
> Fixed: chromium:358427300
> Change-Id: I541166cbd122c7e296ecdd2b5a9f9a107fe82872
> Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5782349
> Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
> Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
> Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Change-Id: Id9f99c21480e7f72617b2907c35356e5033a3ce4
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5782859
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Yuly Novikov <ynovikov@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
|
|
|
b60dc9d5
|
2024-08-12T11:47:16
|
|
Remove the linkage-specification on `main`
According to the standard, the main function should have no
linkage-specification [1]. New version of clang will enforce
this via a warning [2] which causes build failure if building
with `-Werror` which treats warnings as error.
Remove the superfluous linkage-specification to fix the error.
[1]: https://eel.is/c++draft/basic.start#main-3
[2]: https://github.com/llvm/llvm-project/pull/101853
Fixed: chromium:358427300
Change-Id: I541166cbd122c7e296ecdd2b5a9f9a107fe82872
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5782349
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
|
|
d193d51b
|
2024-06-17T22:46:08
|
|
Replace issue ids post migration to new issue tracker
This change replaces anglebug.com/NNNN links.
Bug: None
Change-Id: I8ac3aec8d2a8a844b3d7b99fc0a6b2be8da31761
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/5637912
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Commit-Queue: Shahbaz Youssefi <syoussefi@chromium.org>
|
|
f62405c5
|
2022-10-31T18:47:52
|
|
Add utilities to get the Android backtrace
* Added getBacktraceInfo(), which returns the backtrace information
from the ANGLE code, including the addresses and the symbols if
possible
* Returns the data in an instance of the new class BacktraceInfo.
* In order to access this function, backtrace_utils.h has been
included in vk_utils.h
* New GN flag to make use of this feature:
* angle_enable_unwind_backtrace_support
* Current only available on Android (debug mode)
* If the flag is disabled, getBacktraceInfo() returns an empty
object.
* Added functions in util/ (per platform) to print the BacktraceInfo
data.
* Example of usage:
angle::printBacktraceInfo(angle::getBacktraceInfo());
* Minor edit: Moved cstdint from android_util.cpp to its header.
Bug: b/258475923
Change-Id: I6115462a1a2845d40c7cafc14ce52df09ecdcf34
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/3995843
Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Reviewed-by: Charlie Lao <cclao@google.com>
Commit-Queue: Amirali Abdolrashidi <abdolrashidi@google.com>
|
|
ebf00703
|
2020-12-29T16:25:54
|
|
angle_end2end_tests passes on iOS!
Miscellaneous test skips and fixes for iOS.
Bug: angleproject:5417
Bug: angleproject:5491
Change-Id: Id0785e6243949fc756e4d7923dbbe77a411052f3
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2606656
Reviewed-by: Shahbaz Youssefi <syoussefi@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: James Darpinian <jdarpinian@chromium.org>
|
|
|
0dbe7cdd
|
2020-12-08T12:24:35
|
|
iOS testing support
angle_white_box_tests build and runs and passes on the iOS
simulator with this change. angle_end2end_tests builds and runs but
crashes.
Bug: angleproject:4256
Bug: angleproject:5417
Change-Id: I8817e46415c4598cbfae49804727a2e9b21baff1
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2600361
Commit-Queue: James Darpinian <jdarpinian@chromium.org>
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
|