[Lldb-commits] [PATCH] D129783: [lldb] Skip tests using int128 on ARM

DavidSpickett added a comment. Thanks! The Arm/AArch64 Linux bots were green. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129783/new/ https://reviews.llvm.org/D129783 ___ lldb-commits mailing list lldb

[Lldb-commits] [PATCH] D129783: [lldb] Skip tests using int128 on ARM

werat abandoned this revision. werat added a comment. I have relanded the patch with the fixed tests here -- https://github.com/llvm/llvm-project/commit/7d297de95117e783bbfaecbba1f72fc55de05a30 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129783/n

[Lldb-commits] [PATCH] D129783: [lldb] Skip tests using int128 on ARM

DavidSpickett added a comment. > which disabled the test for both 32-bit and 64-bit That one also runs fine on Linux so probably just overly cautious. I see `lldb/test/API/commands/expression/rdar44436068/Test128BitsInteger.py` as well, that only disables Arm. Once your changes have passed the

[Lldb-commits] [PATCH] D129783: [lldb] Skip tests using int128 on ARM

werat added a comment. I didn't see the test failure on AArch64, so you're both probably right. I've searched the code for other uses of int128 and found a similar place, which disabled the test for both 32-bit and 64-bit -- https://github.com/llvm/llvm-project/blob/30c2406e270cc5dab8da813ce5c5

[Lldb-commits] [PATCH] D129783: [lldb] Skip tests using int128 on ARM

DavidSpickett added a comment. (in case you aren't familiar when we say "Arm" we mean 32 bit Arm and AArch64 refers to 64 bit, it's not always that way but for this purpose it is) I couldn't find a statement that AArch64 supports int128 but it will compile (https://godbolt.org/z/j5dqz4os4) and

[Lldb-commits] [PATCH] D129783: [lldb] Skip tests using int128 on ARM

labath added a comment. I find the aarch64 part surprising, and it seems that the aarch64 bot did not have a problem with this patch (https://lab.llvm.org/buildbot/#/builders/96/builds/26089). The arm part is expected, and most likely applies to all 32 bit architectures (but I think we only ha

[Lldb-commits] [PATCH] D129783: [lldb] Skip tests using int128 on ARM

jasonmolenda added a comment. A very minor point but API tests don't need `mydir = TestBase.compute_mydir(__file__)` any more thanks to https://reviews.llvm.org/D128077 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129783/new/ https://reviews.l

[Lldb-commits] [PATCH] D129783: [lldb] Skip tests using int128 on ARM

werat created this revision. werat added a reviewer: Michael137. Herald added a subscriber: kristof.beyls. Herald added a project: All. werat requested review of this revision. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. Follow-up fix for https://reviews.llvm.org/D81471.