snu-sf / crellvm-tests

0 stars 0 forks source link

Add gvn_icmp test case #36

Closed alxest closed 8 years ago

alxest commented 8 years ago

inputs_full/gvn_icmp에 icmp 관련된 테스트 케이스를 추가합니다. 위치는 llvm_regrssion_tests 는 llvm regression test만 넣고 programs 는 프로그램만 넣는게 맞을 것 같아서 여기로 해봤는데, instcombine/gvn 등등 사이의 구분은 필요할 것 같아서 gvn_icmp 라는 이름으로 넣어봤습니다. 이후에도 계속 gvn_a gvn_b gvn_c 이런 식으로 넣을텐데, 이게 쌓이면 그냥 gvn 이라는 디렉토리로 구분해놓는게 보기 좋을 것 같습니다. test.py 가 디렉토리 depth 2 를 가정하게 되어있으니 (맞나요?), inputs_full 을 instcombine_unit_tests 등으로 바꾸고 gvn_unit_tests/icmp 로 만들어서 넣는게 나을까 싶기도 하고요.

@SanghoonPark 님은 inputs_full/mem2reg 하나 만들어서 거기에 다 넣으려는 것 같은데 그러면 instcombine 도 inputs_full/instcombine 을 만들어서 거기에 다 넣는게 개념적으로 맞을 것 같습니다.

aqjune commented 8 years ago

동감합니다. inputs_full 안에 있는 폴더들을 하나로 합치고 안에 있는 example.ll을 <최적화이름>.ll 로 바꾸겠습니다.