[PATCH] D121556: [randstruct] Add randomize structure layout support

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 8 18:30:47 PDT 2022


MaskRay added a comment.

Windows test is still failing after the std::shuffle => llvm::shuffle change, guess time for revert.

  ../../clang/unittests/AST/RandstructTest.cpp(165): error: Expected equality of these values:
    Expected
      Which is: { "lettuce", "bacon", "mayonnaise", "tomato" }
    getFieldNamesFromRecord(RD)
      Which is: { "bacon", "lettuce", "mayonnaise", "tomato" }
  [  FAILED  ] StructureLayoutRandomization.MarkedRandomize (47 ms)
  [----------] 1 test from StructureLayoutRandomization (47 ms total)
  
  [----------] Global test environment tear-down
  [==========] 1 test from 1 test suite ran. (47 ms total)
  [  PASSED  ] 0 tests.
  [  FAILED  ] 1 test, listed below:
  [  FAILED  ] StructureLayoutRandomization.MarkedRandomize
  
   1 FAILED TEST
  
  ********************
  Testing:  0.. 10.. 20.. 30.. 40.. 50.. 60.. 70.. 80.. 90.. 
  ********************
  Failed Tests (1):
    Clang-Unit :: AST/./ASTTests.exe/StructureLayoutRandomization.MarkedRandomize


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121556



More information about the cfe-commits mailing list