[PATCH] D131914: [ubsan-minimal] Report the address of an error

[Vitaly Buka via Phabricator via llvm-commits]

vitalybuka added a comment.

In D131914#3724102 <https://reviews.llvm.org/D131914#3724102>, @ikudrin wrote:

> Unfortunately, I have no idea how to check the new output. Will be grateful for any suggestions.

Sorry, didn't notice on the first pass.
my favorite way for find this is to brake code around and run all tests: check-all

something like this and "ninja check-ubsan-minimal"

  diff --git a/compiler-rt/test/ubsan_minimal/TestCases/alignment-assumption.c b/compiler-rt/test/ubsan_minimal/TestCases/alignment-assumption.c
  index 08696c81e573..763261dda1d5 100644
  --- a/compiler-rt/test/ubsan_minimal/TestCases/alignment-assumption.c
  +++ b/compiler-rt/test/ubsan_minimal/TestCases/alignment-assumption.c
  @@ -8,7 +8,7 @@ int main(int argc, char *argv[]) {
   
     void *t = __builtin_assume_aligned(ptr + 1, 0x8000);
     (void)t;
  -  // CHECK: ubsan: alignment-assumption
  +  // CHECK: ubsan: alignment-assumption at 0x{{[0-9af]}}
     // CHECK-NOT: alignment-assumption
   
     free(ptr);

maybe update all these few test cases in the dir


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131914/new/

https://reviews.llvm.org/D131914